backend/src/main/java/org/sejongisc/backend/common/auth/config/OAuth2SuccessHandler.java (2)

104-108: localhost를 domain 속성으로 설정하면 쿠키가 제대로 저장되지 않을 수 있습니다.

RFC 6265에 따르면 localhost에는 domain 속성을 설정하지 않는 것이 권장됩니다. 많은 브라우저가 domain=localhost를 거부하거나 일관성 없이 처리할 수 있습니다. 로컬 환경에서는 domain 속성을 아예 생략해야 합니다.

다음과 같이 수정하세요:

-        boolean isProd = Arrays.asList(env.getActiveProfiles()).contains("prod");
-
-        String sameSite = isProd ? "None" : "Lax";
-        boolean secure = isProd;
-        String domain = isProd ? "sisc-web.duckdns.org" : "localhost";
+        boolean isProd = Arrays.asList(env.getActiveProfiles()).contains("prod");
+
+        String sameSite = isProd ? "None" : "Lax";
+        boolean secure = isProd;

그리고 쿠키 생성 시 조건부로 domain을 설정하세요:

         ResponseCookie accessCookie = ResponseCookie.from("access", accessToken)
                 .httpOnly(true)
                 .secure(secure)    // 로컬=false, 배포=true
                 .sameSite(sameSite)  // 로컬= "Lax", 배포="None"
-                .domain(domain)
+                .domain(isProd ? "sisc-web.duckdns.org" : null)  // 로컬에서는 domain 속성 생략
                 .path("/")
                 .maxAge(60L * 60)  // 1 hour
                 .build();

         ResponseCookie refreshCookie = ResponseCookie.from("refresh", refreshToken)
                 .httpOnly(true)
                 .secure(secure)
                 .sameSite(sameSite)
-                .domain(domain)
+                .domain(isProd ? "sisc-web.duckdns.org" : null)  // 로컬에서는 domain 속성 생략
                 .path("/")
                 .maxAge(60L * 60 * 24 * 14) // 2 weeks
                 .build();

---

104-104: 환경 판별 로직이 SecurityConfig와 중복됩니다.

isProd 로직이 SecurityConfig의 isProd() 메서드와 중복됩니다. 또한 구현 방식도 약간 다릅니다 (여기서는 Arrays.asList, SecurityConfig에서는 List.of). 공통 유틸리티 클래스나 설정 빈으로 추출하는 것을 고려하세요.

예시:

@Component
public class EnvironmentHelper {
    private final Environment env;
    
    public EnvironmentHelper(Environment env) {
        this.env = env;
    }
    
    public boolean isProd() {
        return Arrays.asList(env.getActiveProfiles()).contains("prod");
    }
}

그리고 두 클래스 모두에서 주입받아 사용:

+    private final EnvironmentHelper environmentHelper;
     ...
-    boolean isProd = Arrays.asList(env.getActiveProfiles()).contains("prod");
+    boolean isProd = environmentHelper.isProd();